Skip to content

chore: email-messages update shouldn't overwrite by default#71

Open
notnmeyer wants to merge 2 commits intomainfrom
chore/email-messages-update-should-require-explicit-revision
Open

chore: email-messages update shouldn't overwrite by default#71
notnmeyer wants to merge 2 commits intomainfrom
chore/email-messages-update-should-require-explicit-revision

Conversation

@notnmeyer
Copy link
Copy Markdown
Member

@notnmeyer notnmeyer commented May 8, 2026

no overwrite-by-default behavior,

 ➜ task -- email-messages update blah --lmx "<H1>hello</H1>"

   ERROR

  At least one of the flags in the group [expected-revision-id force] is required.

exit status 1
task: Failed to run task "default": exit status 1

revision id is set the same way,

➜ task -- email-messages update blah --lmx "<H1>hello</H1>" -r blah
Updated.
...

the previous behavior (looking up and sending the most recent revision) is behind --force/-f,

➜ task -- email-messages update blah --lmx "<H1>hello</H1>" --force
Updated.
...

you cant use --force and --expected-revision-id together,

➜ task -- email-messages update blah --lmx "<H1>hello</H1>" -r blah -f

   ERROR

  If any flags in the group [expected-revision-id force] are set none of the others can be; [expected-revision-id force] were all set.

@notnmeyer notnmeyer self-assigned this May 8, 2026
@notnmeyer notnmeyer marked this pull request as ready for review May 8, 2026 20:08
@notnmeyer notnmeyer requested a review from chrisfrantz May 8, 2026 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant